Skip to content
This repository has been archived by the owner on Sep 9, 2022. It is now read-only.

Add gh-team-whitelist command line argument #7

Closed
wants to merge 4 commits into from

Conversation

aknysh
Copy link

@aknysh aknysh commented Oct 12, 2018

what

  • Add gh-team-whitelist command line argument

why

  • Comma separated list of key-value pairs representing the GitHub teams and the operations that the members of a particular team are allowed to perform
  • The format is {team}:{command},{team}:{command}, e.g. --gh-team-whitelist=dev:plan,ops:apply,devops:*. This example means to give the users from the dev GitHub team the permissions to execute the plan command, give the ops team the permissions to execute the apply command, and allow the devops team to perform any operation
  • If this argument is not provided, the default value (*:*) will be used and the default behavior will be to not check permissions and to allow users from any team to perform any operation

@aknysh aknysh self-assigned this Oct 12, 2018
.travis.yml Outdated
- make go:deps-dev

script:
- make go:deps

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use /

Makefile Outdated
@@ -86,3 +86,9 @@ end-to-end-tests: ## Run e2e tests

website-dev:
yarn website:dev

go-get:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's use go/get

Makefile Outdated
go-get:
go get

go-build:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's use go/build (e.g. like our docker/build)

@osterman
Copy link

We've merged #9 which does the same thing but uses cloudposse/atlantis go deps.

Copy link

@goruha goruha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aknysh resolve conflicts

@aknysh
Copy link
Author

aknysh commented Jan 7, 2019

this was implemented and merged in #9
closing this PR for now

@aknysh aknysh closed this Jan 7, 2019
@aknysh aknysh deleted the gh-team-whitelist branch April 30, 2019 18:37
@Jaff
Copy link

Jaff commented Jul 31, 2019

Will setting ATLANTIS_GH_TEAM_WHITELIST as environment variable behave as if I used command-line flag --gh-team-whitelist ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants